Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

INSERT has more expressions than target columns (issue #104) #105

Merged
merged 2 commits into from
Feb 10, 2024
Merged

INSERT has more expressions than target columns (issue #104) #105

merged 2 commits into from
Feb 10, 2024

Conversation

zenodeapp
Copy link
Contributor

Fixes issue #104 where too many VALUES were given to the tx_bridge_pool table.

@zenodeapp zenodeapp changed the title NSERT has more expressions than target columns (issue #104) INSERT has more expressions than target columns (issue #104) Feb 8, 2024
@zenodeapp zenodeapp closed this Feb 8, 2024
@zenodeapp zenodeapp reopened this Feb 8, 2024
@Pa1amar
Copy link

Pa1amar commented Feb 8, 2024

thanks, this fixed my error

  2024-02-08T12:13:03.471824Z ERROR namadexer::indexer: Closing block producer task due to an error saving last block: Database error: error returned from database: INSERT has more expressions than target columns
    at src/indexer/mod.rs:229

  2024-02-08T12:13:03.472075Z  INFO namadexer::indexer: Got block 1834
    at src/indexer/mod.rs:51
    in namadexer::indexer::get_block with block_height: 1834

@Pa1amar Pa1amar mentioned this pull request Feb 8, 2024
Closed
@SNSMLN
Copy link

SNSMLN commented Feb 8, 2024 •
edited
Loading

same error

INFO namadexer::indexer: Got block 1836
    at src/indexer/mod.rs:51
    in namadexer::indexer::get_block with block_height: 1836

  2024-02-08T15:57:33.900536Z ERROR namadexer::indexer: Closing block producer task due to an error saving last block: Database error: error returned from database: INSERT has more expressions than target columns
    at src/indexer/mod.rs:229

Error: DB(Database(PgDatabaseError { severity: Error, code: "42601", message: "INSERT has more expressions than target columns", detail: None, hint: None, position: Some(Original(397)), where: None, schema: None, table: None, column: None, data_type: None, constraint: None, file: Some("analyze.c"), line: Some(949), routine: Some("transformInsertRow") }))


git rev-parse HEAD
a324033ab367f6d2bcc66cbb2704aa984d424135

git tag --contains HEAD
v0.1.0-rc.3

@rllola
Copy link
Contributor

rllola commented Feb 8, 2024

On which network did you find this ? It doesn't show on the current shielded-expedition

@Pa1amar
Copy link

Pa1amar commented Feb 8, 2024 •
edited
Loading

On which network did you find this ? It doesn't show on the current shielded-expedition

I see the same on the shielded-expedition.88f17d1d14

  2024-02-08T18:40:51.705055Z ERROR namadexer::indexer: Closing block producer task due to an error saving last block: Database error: error returned from database: INSERT has more expressions than target columns
    at src/indexer/mod.rs:229

  2024-02-08T18:40:51.705174Z  INFO namadexer::indexer: Got block 1834
    at src/indexer/mod.rs:51
    in namadexer::indexer::get_block with block_height: 1834

@zenodeapp
Copy link
Contributor Author

Find it weird that this issue wasn't present in every indexer.

Do you want me to pull and merge the latest?

@zenodeapp
Copy link
Contributor Author

zenodeapp commented Feb 8, 2024 •
edited
Loading

@rllola Merged latest commit, so requires approval to test again. (Think the latest commit fixed one of the workflows?)

@zenodeapp
Copy link
Contributor Author

zenodeapp commented Feb 8, 2024 •
edited
Loading

Reporting back, I completely wiped, pulled the latest commit from the repo and did this PR by hand again and the indexer is at the current block height now. So it all should work after this PR gets merged.

On to the next indexer-halt 😅.

PS: I have create_index on true.

@rllola
Copy link
Contributor

rllola commented Feb 9, 2024

I have checked and it it was indeed left from an old structure. I am running the tests and this should be good to merge!

On to the next indexer-halt 😅.

😅 thank you for your contribution

@zenodeapp
Copy link
Contributor Author

zenodeapp commented Feb 9, 2024 •
edited
Loading

I have checked and it it was indeed left from an old structure. I am running the tests and this should be good to merge!

On to the next indexer-halt 😅.

😅 thank you for your contribution

Haha, you're very welcome! It was just one single line, I wouldn't have known how to tackle bigger bugs since my experience with Rust is so little!

Should thank you for creating/maintaining this @rllola!

Mmm, I want ramen now, now that I saw your name 😴🍜.

@rllola rllola merged commit a99261d into Zondax:main Feb 10, 2024
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rllola rllola rllola approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@zenodeapp @Pa1amar @SNSMLN @rllola